Skip to content

Conversation

@RT-Trace
Copy link

No description provided.

@RT-Trace RT-Trace changed the title rt-trace两个软件包Tunnel和virtual terminal rt-trace三项软件包 Oct 30, 2025
@RT-Trace RT-Trace marked this pull request as draft October 30, 2025 09:13
@RT-Trace
Copy link
Author

需要先合并功能所在的仓库,重新测试后再合并该分支

@RT-Trace RT-Trace marked this pull request as ready for review November 7, 2025 08:30
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This pull request adds two new RT-Thread packages (RT-Tunnel and Virtual-Terminal) and updates the existing RTI package with version 2.0.0 support and improved configuration options.

  • Adds RT-Tunnel package for auxiliary communication with RT-Trace tool
  • Adds Virtual-Terminal package for high-speed virtual terminal functionality
  • Updates RTI package with v2.0.0 support and version-specific configuration options

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
tools/Virtual-Terminal/package.json Package metadata for new Virtual-Terminal package
tools/Virtual-Terminal/Kconfig Configuration options for Virtual-Terminal package
tools/RT-Tunnel/package.json Package metadata for new RT-Tunnel package
tools/RT-Tunnel/Kconfig Configuration options for RT-Tunnel with tunnel number and buffer size settings
tools/Kconfig Adds source directives for new RT-Tunnel and Virtual-Terminal packages
system/rti/package.json Updates RTI with v2.0.0 version, adds "insight" keyword, and updates URL format
system/rti/Kconfig Adds v2.0.0 support with version-specific configuration options

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

string
default "/packages/system/rti"

# 版本选择
Copy link

Copilot AI Nov 7, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The comment uses Chinese characters. For consistency with the rest of the codebase which uses English comments, consider translating to English: '# Version selection'.

Suggested change
# 版本选择
# Version selection

Copilot uses AI. Check for mistakes.
config BOARD_SYSTICK_FREQ_IN_HZ
int "CPU frequency (Hz)"
default 168000000
depends on PKG_USING_RTI_V200 || PKG_USING_RTI_LATEST_VERSION # 仅v2和latest版本可见
Copy link

Copilot AI Nov 7, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The comment uses Chinese characters. For consistency with other comments in the file, consider translating to English: '# Only visible for v2 and latest versions'.

Suggested change
depends on PKG_USING_RTI_V200 || PKG_USING_RTI_LATEST_VERSION # 仅v2和latest版本可见
depends on PKG_USING_RTI_V200 || PKG_USING_RTI_LATEST_VERSION # Only visible for v2 and latest versions

Copilot uses AI. Check for mistakes.
endif

# ----------------------------
# v2版本仅显示CPU频率配置
Copy link

Copilot AI Nov 7, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The comment uses Chinese characters. For consistency with other comments in the file, consider translating to English: '# Only CPU frequency configuration is shown for v2 version'.

Suggested change
# v2版本仅显示CPU频率配置
# Only CPU frequency configuration is shown for v2 version

Copilot uses AI. Check for mistakes.
Required for v2.0.0+ version: CPU clock frequency, used for timestamp calculation.

# ----------------------------
# v1版本显示原有其他配置
Copy link

Copilot AI Nov 7, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The comment uses Chinese characters. For consistency with other comments in the file, consider translating to English: '# Original configurations shown for v1 version'.

Suggested change
# v1版本显示原有其他配置
# Original configurations shown for v1 version

Copilot uses AI. Check for mistakes.
# ----------------------------
# v1版本显示原有其他配置
# ----------------------------
if PKG_USING_RTI_V100 # 仅v1.0.0版本可见
Copy link

Copilot AI Nov 7, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The comment uses Chinese characters. For consistency with other comments in the file, consider translating to English: '# Only visible for v1.0.0 version'.

Suggested change
if PKG_USING_RTI_V100 # 仅v1.0.0版本可见
if PKG_USING_RTI_V100 # Only visible for v1.0.0 version

Copilot uses AI. Check for mistakes.
@Rbb666 Rbb666 changed the title rt-trace三项软件包 添加rt-trace相关软件包 Nov 7, 2025
@Rbb666 Rbb666 merged commit 0c567d7 into RT-Thread:master Nov 7, 2025
17 of 18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants